Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add definitions for afunix.h #968

Open
wants to merge 1 commit into
base: 0.3
Choose a base branch
from
Open

add definitions for afunix.h #968

wants to merge 1 commit into from

Conversation

fanzeyi
Copy link

@fanzeyi fanzeyi commented Dec 23, 2020

Since Windows 17063, there is UNIX domain socket support in Windows. Along with the support afunix.h is added to define SOCKADDR_UN struct for creating UNIX domain sockets.

This pull request adds the definition mirrors afunix.h in Windows SDK.

See "AF_UNIX comes to Windows"

Windows header file location: C:\Program Files (x86)\Windows Kits\10\Include\10.0.19041.0\shared\afunix.h

@Thomasdezeeuw
Copy link

Any updates on this? We need this in socket2: rust-lang/socket2#249.

@fanzeyi
Copy link
Author

fanzeyi commented Aug 9, 2021

Not sure what I can do to get this land. winapi-rs seems to be completely stagnated now. :(

There was some discussion about moving winapi-rs to the nursery but nothing has happened it seems.

@@ -0,0 +1,10 @@
use shared::minwindef::DWORD;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice! I came up with practically the same patch without knowing you add a MR.

I think you should add the common top header:

// Licensed under the Apache License, Version 2.0
// <LICENSE-APACHE or http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your option.
// All files in the project carrying such notice may not be copied, modified, or distributed
// except according to those terms.
//! Definitions for the AF_UNIX socket address family.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants